Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add symengine #238 #241

Closed
wants to merge 4 commits into from
Closed

Conversation

zasexton
Copy link
Contributor

This addresses the issue for adding the optional symengine and exprtk.h thirdparty libraries to svFSIplus

Current situation

Refer to #238 for motivating reason for these additions

Release Notes

This pull request adds an optional cmake argument "SV_USE_SYMENGINE" to optionally fetch and compile the third party library at compilation time.

Documentation

readme documentation has been updated for the exprtk.h library since the header file was added to the ThirdParty folder.

Testing

Note that this pull request merely makes the symengine and exprtk.h libraries available. There is no added functionality within svFSIplus based on these libraries yet. This pull request is to verify and obtain agreement on the code architecture for adding these optional libraries.

Code of Conduct & Contributing Guidelines

@zasexton zasexton self-assigned this Aug 20, 2024
@zasexton zasexton marked this pull request as draft August 20, 2024 19:38
@zasexton zasexton requested a review from ktbolt August 20, 2024 20:56
@ktbolt
Copy link
Collaborator

ktbolt commented Aug 21, 2024

@zasexton I cloned your add-symengine-#238 branch and the compile failed on MacOS 13.6.9 M2 with ld: library not found for -lsymengine

And I don't see the odes.h and odes.cpp files.

@zasexton zasexton closed this Sep 30, 2024
@zasexton
Copy link
Contributor Author

I've closed this pull request as it was a draft for the eventual incorporation of a symbolic engine. This work is still underway and will be included in a series of pull requests related to #218

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants